lib/crewai/src/crewai/governance/__init__.py (1)

1-2: ⚡ Quick win

Export types from __init__.py for better ergonomics.

The governance module's __init__.py is empty, requiring users to import from the full submodule path. Exporting the public types would make imports more convenient and discoverable.

📦 Proposed export pattern

+"""
+CrewAI governance contracts.
+"""
+
+from crewai.governance.governance_decision import (
+    GovernanceDecision,
+    GovernanceOutcome,
+)
+
+__all__ = ["GovernanceDecision", "GovernanceOutcome"]

This allows cleaner imports:

from crewai.governance import GovernanceDecision, GovernanceOutcome

instead of:

from crewai.governance.governance_decision import GovernanceDecision, GovernanceOutcome

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@lib/crewai/src/crewai/governance/__init__.py` around lines 1 - 2, Add public
exports to the governance package by importing and re-exporting the key types in
the package __init__.py: import GovernanceDecision and GovernanceOutcome (and
any other public classes) from their defining module (e.g., governance_decision)
and add them to the package's __all__ so users can do "from crewai.governance
import GovernanceDecision, GovernanceOutcome"; ensure you reference the exact
class names GovernanceDecision and GovernanceOutcome and any other public
symbols you want exposed.

test_governance_decision_contract.py (1)

98-103: ⚡ Quick win

Consider adding error outcome fixture for completeness.

The current outcome fixture only tests outcome="executed". Since GovernanceOutcome defines error_type and error_message fields and includes "error" as a valid outcome, adding a fixture that demonstrates the error case would strengthen contract coverage.

🧪 Proposed error outcome fixture

FIXTURE_OUTCOME_ERROR: GovernanceOutcome = {
    "decision_id": "d-002",  # Link to FIXTURE_DENY or any decision
    "outcome": "error",
    "error_type": "PermissionError",
    "error_message": "Insufficient permissions to execute tool",
    "completed_at": "2026-06-03T14:01:05Z",
}

Then add to test_all_fixtures_json_serializable:

     fixtures: list[dict[str, Any]] = [
         FIXTURE_ALLOW,
         FIXTURE_DENY,
         FIXTURE_REQUIRE_APPROVAL,
         FIXTURE_ALLOW_WITH_EXTENSION,
         FIXTURE_REVISE,
         FIXTURE_OUTCOME,
+        FIXTURE_OUTCOME_ERROR,
         FIXTURE_UNKNOWN_EXTENSION,
     ]

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@test_governance_decision_contract.py` around lines 98 - 103, Add an
error-case fixture for GovernanceOutcome named FIXTURE_OUTCOME_ERROR that sets
"outcome" to "error" and includes "error_type" and "error_message" plus
"decision_id" and "completed_at" to mirror FIXTURE_OUTCOME structure; then
include FIXTURE_OUTCOME_ERROR in the test_all_fixtures_json_serializable
assertions so the error-path (outcome="error") is covered by serialization tests
(reference symbols: FIXTURE_OUTCOME, FIXTURE_OUTCOME_ERROR, GovernanceOutcome,
test_all_fixtures_json_serializable).